Skip to content

Update to use the new NVKS runners #426

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Jan 30, 2025
Merged

Update to use the new NVKS runners #426

merged 1 commit into from
Jan 30, 2025

Conversation

leofang
Copy link
Member

@leofang leofang commented Jan 30, 2025

The old runners are removed, so all CIs are blocked due to lack of runners.

@leofang leofang added P0 High priority - Must do! CI/CD CI/CD infrastructure labels Jan 30, 2025
@leofang leofang added this to the cuda.core beta 3 milestone Jan 30, 2025
@leofang leofang self-assigned this Jan 30, 2025
@copy-pr-bot copy-pr-bot
Copy link
Contributor

copy-pr-bot bot commented Jan 30, 2025

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@leofang
Copy link
Member Author

leofang commented Jan 30, 2025

/ok to test

vzhurba01
vzhurba01 approved these changes Jan 30, 2025
View reviewed changes
Copy link
Collaborator

@vzhurba01 vzhurba01 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So that's what was happening 👍

@leofang leofang enabled auto-merge January 30, 2025 17:10
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@leofang
Copy link
Member Author

leofang commented Jan 30, 2025

Since both H100 and doc build CIs pass, let me admin-merge this to save some CI resources.

@leofang leofang disabled auto-merge January 30, 2025 17:24
@leofang leofang merged commit 74379bb into NVIDIA:main Jan 30, 2025
69 of 70 checks passed
@leofang leofang deleted the use_new_runner branch January 30, 2025 17:25
@github-actions GitHub Actions
Copy link

Doc Preview CI
Preview removed because the pull request was closed or merged.

@leofang leofang mentioned this pull request Jan 31, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vzhurba01 vzhurba01 vzhurba01 approved these changes

@ksimpson-work ksimpson-work ksimpson-work approved these changes

Assignees

@leofang leofang

Labels
CI/CD CI/CD infrastructure P0 High priority - Must do!
Projects
None yet
Milestone
cuda.core beta 3
Development

Successfully merging this pull request may close these issues.
3 participants
@leofang @vzhurba01 @ksimpson-work